Skip to content
This repository was archived by the owner on Oct 6, 2026. It is now read-only.

Fix cross-version links in v25.2 documentation - #22305

Open
ebembi-crdb wants to merge 3 commits into
mainfrom
fix-cross-version-links-v25.2
Open

ebembi-crdb wants to merge 3 commits into
mainfrom
fix-cross-version-links-v25.2

Conversation

@ebembi-crdb

Copy link
Copy Markdown
Contributor

Summary

  • Fix cross-version link violations in v25.2 documentation
  • Replace hardcoded version references with {{ page.version.version }}

Changes

  • 101 files modified
  • 266 cross-version links fixed

Test Plan

  • Cross-version link linter passes for v25.2
  • Jekyll build completes successfully

@netlify

netlify Bot commented Jan 22, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit 293bb60
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-api-docs/deploys/697a1988f93ce20008599615

@github-actions

github-actions Bot commented Jan 22, 2026 •

Copy link
Copy Markdown

Files changed:

@netlify

netlify Bot commented Jan 22, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit 293bb60
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-interactivetutorials-docs/deploys/697a1988f3d2ef00073f1481

@netlify

netlify Bot commented Jan 22, 2026 •

Copy link
Copy Markdown

✅ Netlify Preview

Name Link
🔨 Latest commit 293bb60
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-docs/deploys/697a1988c8ef0800086c780a
😎 Deploy Preview https://deploy-preview-22305--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

ebembi-crdb added 2 commits January 27, 2026 16:53
Replace broken Liquid syntax with working path format:
- From: {{ 'images/{{ page.version.version }}/...' | relative_url }}
- To: /docs/images/{{ page.version.version }}/...
Copy changefeed-structure.png from v25.1 and fix image path
to use proper Liquid syntax with relative_url filter.
@rmloveland
rmloveland self-requested a review January 29, 2026 19:26
`vehicle_location_histories` | Vehicle location history.

<img src="{{ 'images/v24.2/movr-schema.png' | relative_url }}" alt="Geo-partitioning schema" style="max-width:100%" />
<img src="/docs/images/{{ page.version.version }}/movr-schema.png" alt="Geo-partitioning schema" style="max-width:100%" />

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ebembi-crdb (cc @taroface FYI) the fact this works and passes build checks seems to invalidate my theory that the "new non-version-specific link syntax" shown here doesn't work in include files, since it apparently does (good news!)

@rmloveland rmloveland left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

This branch has not been deployed

No deployments
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants